Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: api 연결 #15

Merged
merged 50 commits into from
Sep 3, 2024
Merged

feat: api 연결 #15

merged 50 commits into from
Sep 3, 2024

Conversation

iOdiO89
Copy link
Member

@iOdiO89 iOdiO89 commented Sep 1, 2024

정렬이랑 탭별로 출력해야 하는 기준들이 조금씩 달라서 정리했습니다!
의도랑 구현한 거랑 일치하는지 위주로 봐주시면 좋을 것 같습니다

승인대기

  • 대상: orderhistory의 status = PENDING 인 order
  • 정렬: 각 order의 createdAt을 기준으로 최신순 정렬
  • order GET: 5초마다 refetch
    ㄴ 이전 fetch의 승인대기탭 order count < 현재 fetch의 order count 일 때, 효과음 발생
    ㄴ 새로 로그인해서 들어왔을 때 승인대기 중인 주문이 존재한다면 효과음 발생

진행 중

  • 대상: status = PROCESS인 paymenthistory 내에서 orderhistory의 status = APPROVED 인 것
  • 정렬: payment 내에서 가장 최근에 approved된 order의 updatedAt을 기준으로 최신순 정렬

히스토리

  • 대상: status = HISTORY인 paymenthistory
  • 정렬: paymenthistory의 paidAt 기준으로 오래된 순으로 정렬

api notice

  1. PATCH api를 통해 payment history status를 HISTORY로 바꾸었을 때 paidAt이 계속 null값으로 남아있습니다.
  2. 서버시간이 KST가 아니라 UTC라서 히스토리 탭에서 filter 걸었을 때 KST 기준으로 filter가 걸리는 건지 확인해주시면 좋을 것 같아요. (1번 문제 때문에 프론트에서 확인이 안됩니다 ㅠ)
  3. 주문 거절하기 api 500에러가 발생합니다 (병선님께 전달 드렸어요)

추가)
주문 승인 되었을 때 사라지는 애니메이션 같은 게 필요할까요? 승인대기 버튼 눌렀을 때 아무런 문구 없이 바로 승인대기탭에서 사라지니까 원하는 주문이 제대로 승인되었는지 알기 어려운 것 같습니다.

iOdiO89 added 30 commits August 23, 2024 02:41
@iOdiO89 iOdiO89 added the feature New feature or request label Sep 1, 2024
@iOdiO89 iOdiO89 self-assigned this Sep 1, 2024
@DeveloperRyou
Copy link
Contributor

효과음 발생 관련하여서, 승인대기 중인 주문이 존재한다면 계속 효과음 발생으로 수정 부탁드립니다! (GET 했을 때 승인대기 주문이 있다면 그냥 계속 효과음 발생)

PATCH api 관련해서는

  1. 결제완료 (DONE) 시 PaidAt 안찍힘
  2. 거절하기 시 500 에러 발생

이 문제가 되는 상황인거죠?

Copy link
Contributor

@DeveloperRyou DeveloperRyou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드는 좋은것 같습니다.
다만, 한 pr에 많은 커밋이 있어서 디테일하게 확인하기는 어려웠어요!
쪼개서 올려주시면 다음에는 더 좋을것 같습니다~

@iOdiO89
Copy link
Member Author

iOdiO89 commented Sep 2, 2024

  • 승인대기중인 주문이 1개 이상 있다면 항상 효과음 발생하는 거로 수정하란 말씀이시죠? 수정하겠습니다!
  • api 에러 두 가지 다 맞습니다! 2번은 병선님께 전달드리기는 했는데 아직 해결됐다는 연락은 못 받았어요.
  • 쪼개서 올리려고 했었는데 중간중간 api가 수정되고 카카오로그인도 붙이고 하다보니까 길어졌네요 ㅠㅠㅠ 다음부터는 쪼개서 올리겠습니다!

@iOdiO89 iOdiO89 linked an issue Sep 3, 2024 that may be closed by this pull request
1 task
@iOdiO89 iOdiO89 merged commit e2a3052 into the-kingdoms:develop Sep 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FE 주문 정보 api를 연결한다
2 participants